Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

workflows/triage: skip recursive dependents for systemd #201157

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cho-m
Copy link
Member

@cho-m cho-m commented Dec 14, 2024

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

We have defined all dependents with linkage to systemd as direct based on a prior run https://github.com/Homebrew/homebrew-core/actions/runs/12086331739/job/33706038266 (linkage failures are for other formulae).

Recursive dependents bring testing time up to 1d10h so would help to only test the direct dependents.

Not sure if there are any runtime dlopen or systemd executables we need to worry about. Though part of goal for the indirect dependencies check is to make the CI-skip-recursive-dependents the default behavior anyways.

@cho-m cho-m requested a review from a team as a code owner December 14, 2024 15:04
@cho-m cho-m added the maintainer feedback Additional maintainers' opinions may be needed label Dec 14, 2024
@github-actions github-actions bot added automerge-skip `brew pr-automerge` will skip this pull request workflows PR modifies GitHub Actions workflow files labels Dec 14, 2024
@cho-m cho-m force-pushed the systemd-skip-recursive branch from 4374400 to ea0fb91 Compare December 14, 2024 15:17
Copy link
Member

@chenrui333 chenrui333 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sense 👍

@cho-m
Copy link
Member Author

cho-m commented Dec 14, 2024

Also removing self-hosted Linux. There are only 50 formulae with dependency and prior runs worked.

I think recursive run hits around 500 formulae.

@cho-m cho-m removed the maintainer feedback Additional maintainers' opinions may be needed label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge-skip `brew pr-automerge` will skip this pull request workflows PR modifies GitHub Actions workflow files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants